Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[fixed]: IPv6 destination address with mixed alphabets did not match. #454

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[fixed]: IPv6 destination address with mixed alphabets did not match. #454

wants to merge 2 commits into from

Conversation

kunitake
Copy link

SUMMARY

IPv6 destination address with only ::/0 or numbers was matched, but IPv6 destination address with mixed alphabets was not matched because there was an omission in the regular expression.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

arista.eos.eos_static_routes module

ADDITIONAL INFORMATION
  • Case that could have been handled
ipv6 route ::/0 2001:db8::1
  • Case that could have not been handled
ipv6 route 2001:db8::/48 2001:db8::1

@kunitake kunitake temporarily deployed to ack September 15, 2023 02:45 — with GitHub Actions Inactive
@kunitake kunitake temporarily deployed to ack September 15, 2023 02:49 — with GitHub Actions Inactive
@softwarefactory-project-zuul
Copy link

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/8da3f0ebc6574d18ba8559e48e01cb2c

✔️ ansible-test-network-integration-eos-httpapi-python39-stable215 SUCCESS in 40m 02s
✔️ ansible-test-network-integration-eos-httpapi-python39-stable214 SUCCESS in 39m 03s
✔️ ansible-test-network-integration-eos-httpapi-python39-stable213 SUCCESS in 35m 29s
✔️ ansible-test-network-integration-eos-httpapi-python39-stable212 SUCCESS in 35m 09s
✔️ ansible-test-network-integration-eos-httpapi-python39-stable211 SUCCESS in 35m 08s
✔️ ansible-test-network-integration-eos-network_cli-python39-stable215-scenario01 SUCCESS in 31m 53s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-python39-stable215-scenario02 SUCCESS in 33m 49s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-libssh-python39-stable215-scenario01 SUCCESS in 31m 52s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-libssh-python39-stable215-scenario02 SUCCESS in 34m 16s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-python39-stable214-scenario01 SUCCESS in 31m 42s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-python39-stable214-scenario02 SUCCESS in 39m 34s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-libssh-python39-stable214-scenario01 SUCCESS in 37m 25s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-libssh-python39-stable214-scenario02 SUCCESS in 40m 02s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-python39-stable213-scenario01 SUCCESS in 31m 26s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-python39-stable213-scenario02 SUCCESS in 40m 30s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-libssh-python39-stable213-scenario01 SUCCESS in 32m 18s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-libssh-python39-stable213-scenario02 SUCCESS in 32m 45s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-python39-stable212-scenario01 SUCCESS in 31m 32s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-python39-stable212-scenario02 SUCCESS in 38m 37s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-libssh-python39-stable212-scenario01 SUCCESS in 31m 40s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-libssh-python39-stable212-scenario02 SUCCESS in 36m 45s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-python39-stable211-scenario01 SUCCESS in 31m 33s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-python39-stable211-scenario02 SUCCESS in 38m 53s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-libssh-python39-stable211-scenario01 SUCCESS in 31m 16s (non-voting)
✔️ ansible-test-network-integration-eos-network_cli-libssh-python39-stable211-scenario02 SUCCESS in 39m 00s (non-voting)
✔️ build-ansible-collection SUCCESS in 9m 26s
✔️ ansible-tox-linters SUCCESS in 11m 39s
✔️ ansible-galaxy-importer SUCCESS in 4m 50s

@rohitthakur2590
Copy link
Contributor

@kunitake Thank you for the contribution, could you please add a changelog and unit/integration tests.

@rohitthakur2590 rohitthakur2590 added the static_routes static routes resource module label Oct 9, 2023
@rohitthakur2590 rohitthakur2590 temporarily deployed to ack October 9, 2023 10:48 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rohitthakur2590 rohitthakur2590 Awaiting requested review from rohitthakur2590

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
static_routes static routes resource module
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kunitake @rohitthakur2590